Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Airlock Refactor and Better Naming for Doors In General, Also Firelocks [MDB IGNORE] #521

Merged
merged 62 commits into from
Mar 31, 2024

Conversation

RimiNosha
Copy link
Contributor

@RimiNosha RimiNosha commented Jan 9, 2024

UI and helper icons stolen from citadel.

About The Pull Request

Fuck me sideways with a rusty knife, this tested my patience. Anyways, yeah, shitcode obliterated and replaced with my own shitcode that's 100% less likely to randomly hang the server when an airlock gets blown up. You know who you are.

Important terminology changes:

  • Airlock doors are now called bulkheads or doors. They're called bulkheads in code now.
  • Firelocks are now called fireheads or firedoors. The're called fireheads in code now.
  • Magic airlocks (the ones with the arrow helpers, are called as such now.
  • Airlocks that use controllers are now just airlocks.
  • Airlocks that are integrated into hallways, and start open are now called firelocks.

Anyways, you no longer have to fuck around with manually setting vars and subtypes for making airlocks. Instead, it's put together with mapping helpers, like citadel.

How to assemble your own airlock is documented in the readme,

TODO:

  • Figure out a better way of achieving magiclocks.

How Does This Help Gameplay?

Makes it clear who's talking about what, and makes it so old firelocks aren't going to be as omnipresent. I've future plans for these things that'll entirely remove them outside of large rooms.

Also makes the station more fire and atmos breakage resistant by having cubbies with their own air-maintaining measures scattered around. This is a good thing with ZAS, trust me.

How Does This Help Roleplay?

More immersive ship and station designs, and proper names for everything.

Proof of Testing

Screenshots/Videos

In your fucking dreams, this shit's broke as fuck thanks to my experiments.

Changelog

🆑
add: Firelocks, which are airlocks that auto-deploy when air becomes unsafe.
qol: Airlock doors and firelocks have been renamed.
refactor: Entirely overhauled how airlocks are handled and named. See pr 521 for more info.
/:cl:

@github-actions github-actions bot added the Tools Oh great, another thing with external dependencies that'll break in the next six months. label Feb 24, 2024
@RimiNosha RimiNosha removed the Merge Conflict Uh oh, a maint merged a PR that edits the same stuff as yours! label Feb 24, 2024
@RimiNosha
Copy link
Contributor Author

Oh fuck

@RimiNosha RimiNosha changed the title Airlock Refactor and Better Naming for Doors In General, Also Firelocks Airlock Refactor and Better Naming for Doors In General, Also Firelocks [MDB IGNORE] Feb 24, 2024
Copy link
Contributor

github-actions bot commented Mar 4, 2024

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself.

@github-actions github-actions bot added the Stale "Anyone home?" label Mar 4, 2024
@RimiNosha RimiNosha removed the Stale "Anyone home?" label Mar 4, 2024
Copy link
Contributor

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself.

@github-actions github-actions bot added the Stale "Anyone home?" label Mar 12, 2024
@RimiNosha RimiNosha removed the Stale "Anyone home?" label Mar 14, 2024
Copy link
Contributor

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself.

@github-actions github-actions bot added the Stale "Anyone home?" label Mar 22, 2024
@github-actions github-actions bot closed this Mar 29, 2024
@RimiNosha RimiNosha reopened this Mar 29, 2024
@RimiNosha RimiNosha removed the Stale "Anyone home?" label Mar 29, 2024
@RimiNosha RimiNosha marked this pull request as ready for review March 30, 2024 21:51
@RimiNosha RimiNosha merged commit 7e8afd7 into Artea-Station:master Mar 31, 2024
17 of 19 checks passed
Artea-Station-Bot added a commit that referenced this pull request Mar 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature How I broke the server in one easy step! Mapping Something something someone placed three reinforced window spawners in one place. Quality of Life Also known as "I deleted a ton of other things and made one particular thing the only way". Refactor Also known as "I added a ton of new code that no one will ever use, while failing CI randomly". Sprites I like pretty things. Tools Oh great, another thing with external dependencies that'll break in the next six months. UI Also known as "I like lagspikes, so I added TGUI to your TGUI".
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant